Skip to content

Feat: Update component dropdown when adding/removing a component - #1159

Merged
alejandralevy merged 9 commits into
mainfrom
feat/update-components-list
Jul 22, 2025
Merged

Feat: Update component dropdown when adding/removing a component#1159
alejandralevy merged 9 commits into
mainfrom
feat/update-components-list

Conversation

@alejandralevy

@alejandralevy alejandralevy commented Jul 16, 2025

Copy link
Copy Markdown
Contributor
  • Fixed dropdown component state updates: components now properly reflect when they're added/removed from entities
  • Disabled instead of hidden: Components shown as disabled in dropdown instead of hidden
  • Auto-remove Visibility component: Automatically removes Visibility when GLTF or MeshCollider is deleted and the other one does no exist
  • Remove Attach Avatar component from the list, as there's no UI component to modify it
Screen.Recording.2025-07-16.at.22.58.55.mov

@github-actions

github-actions Bot commented Jul 16, 2025

Copy link
Copy Markdown
Contributor

Test this pull request

  • The @dcl/sdk package can be tested in scenes by running

    npm install "https://sdk-team-cdn.decentraland.org/@dcl/js-sdk-toolchain/branch/feat/update-components-list/dcl-sdk-7.8.22-16441802101.commit-87a0780.tgz"
  • To test with npx init

    export SDK_COMMANDS="https://sdk-team-cdn.decentraland.org/@dcl/js-sdk-toolchain/branch/feat/update-components-list/dcl-sdk-commands-7.8.22-16441802101.commit-87a0780.tgz"
    npx $SDK_COMMANDS init
  • The @dcl/inspector package can be tested by visiting this url

    • Or by installing it via NPM
    npm install "https://sdk-team-cdn.decentraland.org/@dcl/js-sdk-toolchain/branch/feat/update-components-list/@dcl/inspector/dcl-inspector-7.8.22-16441802101.commit-87a0780.tgz"
  • The /changerealm command to test test in-world

    /changerealm https://sdk-team-cdn.decentraland.org/ipfs/feat/update-components-list-e2e
    
  • You can preview this build entering:
    https://playground.decentraland.org/?sdk-branch=feat/update-components-list

@alejandralevy
alejandralevy force-pushed the feat/update-components-list branch from 79f19ad to aae5087 Compare July 16, 2025 19:16
@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Jul 16, 2025

Copy link
Copy Markdown

Deploying js-sdk-toolchain with  Cloudflare Pages  Cloudflare Pages

Latest commit: 0bf39dd
Status: ✅  Deploy successful!
Preview URL: https://fd7a1ddb.js-sdk-toolchain.pages.dev
Branch Preview URL: https://feat-update-components-list.js-sdk-toolchain.pages.dev

View logs

@codecov

codecov Bot commented Jul 16, 2025

Copy link
Copy Markdown

Codecov Report

Attention: Patch coverage is 10.34483% with 52 lines in your changes missing coverage. Please review.

Project coverage is 61.84%. Comparing base (63d0108) to head (2d44f3a).
Report is 6 commits behind head on main.

Files with missing lines Patch % Lines
...@dcl/inspector/src/hooks/sdk/useEntityComponent.ts 10.34% 52 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1159      +/-   ##
==========================================
- Coverage   62.00%   61.84%   -0.16%     
==========================================
  Files         621      621              
  Lines       20903    20965      +62     
  Branches     2536     2536              
==========================================
+ Hits        12960    12966       +6     
- Misses       7501     7557      +56     
  Partials      442      442              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Comment on lines +152 to +154
if (!isDisabled) {
return { text: description, ...(link && { link }) }
}

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this returned object is the same as the default one... should we send another property here?

@alejandralevy
alejandralevy force-pushed the feat/update-components-list branch from 32fca88 to 0bf39dd Compare July 22, 2025 10:34
@alejandralevy
alejandralevy merged commit 9ae2f83 into main Jul 22, 2025
9 checks passed
@alejandralevy
alejandralevy deleted the feat/update-components-list branch July 22, 2025 10:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants